Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change: only run create_local_resource in the browser #1777

Merged
merged 3 commits into from
Sep 23, 2023
Merged

change: only run create_local_resource in the browser #1777

merged 3 commits into from
Sep 23, 2023

Conversation

gbj
Copy link
Collaborator

@gbj gbj commented Sep 23, 2023

This is one of those changes that is somewhere between a "fix" and a "breaking change." Local resources were intended to run only in the browser, but in fact were running on both the server and in the browser, but without being serialized/shared between the two. This aligns them with the document behavior of only running in the browser.

(Does this fix #1775?)

@akarras
Copy link
Contributor

akarras commented Sep 23, 2023

Can confirm this PR fixes #1775, nice little fix.

@gbj gbj merged commit ae986e7 into main Sep 23, 2023
@gbj gbj deleted the 1775 branch September 23, 2023 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

spawn_local panic within generate_route_list using axum
2 participants